Skip to content

934: add haoking to /api - #899

Merged
luoh00 merged 1 commit into
mainfrom
934
Apr 16, 2026
Merged

934: add haoking to /api#899
luoh00 merged 1 commit into
mainfrom
934

Conversation

@luoh00

@luoh00 luoh00 commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

934

Description of changes

Added "Haoking Luo" to /api endpoint

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

image

Staging

image

@github-actions

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions

Copy link
Copy Markdown
Contributor

Title

934: add haoking to /api


PR Type

Enhancement


Description

  • Adds "Haoking Luo" to server metadata.

  • Updates API's author list.


File Walkthrough

Relevant files
Enhancement
ServerMetadataObject.java
Update server metadata authors list                                           

src/main/java/org/patinanetwork/codebloom/utilities/ServerMetadataObject.java

  • Added "Haoking Luo" to the authors list within the
    ServerMetadataObject constructor.
+7/-1     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@luoh00

luoh00 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor Author

/deploy

@github-actions

Copy link
Copy Markdown
Contributor

Title

934: add haoking to /api


PR Type

Enhancement


Description

  • Added 'Haoking Luo' to authors.

  • Updated server metadata response.


File Walkthrough

Relevant files
Configuration changes
ServerMetadataObject.java
Update server metadata authors                                                     

src/main/java/org/patinanetwork/codebloom/utilities/ServerMetadataObject.java

  • Added Haoking Luo to the list of authors.
  • Updated the ServerMetadataObject constructor to include the new
    author.
+7/-1     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@github-actions

Copy link
Copy Markdown
Contributor

Title

934: add haoking to /api


PR Type

Enhancement


Description

  • Add 'Haoking Luo' to server authors.

  • Update server metadata object.


Diagram Walkthrough

flowchart LR
  A["ServerMetadataObject.java"] -- "Add 'Haoking Luo' to authors" --> B["Updated Server Metadata"]
Loading

File Walkthrough

Relevant files
Enhancement
ServerMetadataObject.java
Add 'Haoking Luo' to server authors list                                 

src/main/java/org/patinanetwork/codebloom/utilities/ServerMetadataObject.java

  • Added "Haoking Luo" to the authors ArrayList in the constructor.
  • This updates the list of contributors associated with the server
    metadata.
+7/-1     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@luoh00

luoh00 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

/deploy

@luoh00
luoh00 merged commit 6e3007a into main Apr 16, 2026
33 checks passed
@luoh00
luoh00 deleted the 934 branch April 16, 2026 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants